Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fail on the s390x architecture with a not supported msg #96

Merged
merged 2 commits into from
Apr 16, 2024

Conversation

spetrosi
Copy link
Contributor

@spetrosi spetrosi commented Apr 12, 2024

Enhancement: Fail on the s390x architecture with a not supported msg

Reason: the bootloader role does not support s390x architecture because s390x uses a different bootloader - not grub2. Currently, the role fails on s390x with grub2-tools rpm not available.

Result: The role fails with a clear error message that s390x is not supported.

Issue Tracker Tickets (Jira or BZ if any): https://issues.redhat.com/browse/SYSROLES-29

@spetrosi spetrosi requested a review from richm as a code owner April 12, 2024 09:25
Copy link

codecov bot commented Apr 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.43%. Comparing base (4d9573a) to head (60c0351).

Additional details and impacted files
@@            Coverage Diff            @@
##           main      #96       +/-   ##
=========================================
+ Coverage      0   78.43%   +78.43%     
=========================================
  Files         0        2        +2     
  Lines         0      255      +255     
=========================================
+ Hits          0      200      +200     
- Misses        0       55       +55     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@spetrosi spetrosi force-pushed the s390x-incompatible branch from c24a4c5 to 3614220 Compare April 12, 2024 09:29
@spetrosi
Copy link
Contributor Author

[citest]

@spetrosi spetrosi force-pushed the s390x-incompatible branch 3 times, most recently from 9cf5a98 to caa9168 Compare April 12, 2024 14:03
@spetrosi spetrosi force-pushed the s390x-incompatible branch from caa9168 to 37cf730 Compare April 12, 2024 14:03
@spetrosi
Copy link
Contributor Author

[citest]

@spetrosi
Copy link
Contributor Author

[citest]

@spetrosi spetrosi force-pushed the s390x-incompatible branch from 1d39450 to 60c0351 Compare April 12, 2024 16:25
@spetrosi
Copy link
Contributor Author

[citest]

@spetrosi spetrosi merged commit 97aed46 into linux-system-roles:main Apr 16, 2024
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants